Skip to content

Conversation

nzaytsev
Copy link
Contributor

@nzaytsev nzaytsev commented Aug 29, 2024

Description

Screen.Recording.2024-08-29.at.16.09.54.mov

Checklist

  • I have followed the guidelines in the Contributing document
  • My changes follow the coding style of this project
  • My changes build without any errors or warnings
  • My changes have been formatted and linted
  • My changes include any required corresponding changes to the documentation (including CHANGELOG.md and README.md)
  • My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
  • My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

@nzaytsev nzaytsev requested a review from eamodio August 29, 2024 09:16
@nzaytsev nzaytsev linked an issue Sep 9, 2024 that may be closed by this pull request
@nzaytsev nzaytsev force-pushed the features/GLVSC-636_add-a-create-branch-split-button-to-the-graph branch from 478ba78 to 6f41d91 Compare September 9, 2024 09:36
@d13 d13 requested review from d13 and removed request for d13 September 27, 2024 14:53
@d13 d13 force-pushed the features/GLVSC-636_add-a-create-branch-split-button-to-the-graph branch 2 times, most recently from a842537 to a291f0b Compare October 2, 2024 17:00
Copy link
Member

@eamodio eamodio left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: but note the testing comment

Comment on lines +158 to +161
if (args.state.flags != null) {
counter++;
}

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm a little concerned about this -- since it could impact a bunch of flows, so will need to be tested.

Comment on lines +307 to +320
.codicon-custom-git-branch-create {
&:before {
content: '\ea68';
}
&:after {
content: '\ea60';
position: absolute;
right: 0;
bottom: 0;
font-size: 0.6em;
line-height: normal;
transform: translate(-50%, 0%);
}
}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is fine for here, but a new GlIcon that we can use in other places would be nice.

@d13 d13 force-pushed the features/GLVSC-636_add-a-create-branch-split-button-to-the-graph branch from b47e03a to 1091131 Compare October 2, 2024 21:41
@d13 d13 merged commit 754ac4e into main Oct 2, 2024
@eamodio eamodio deleted the features/GLVSC-636_add-a-create-branch-split-button-to-the-graph branch October 14, 2024 14:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add a Create Branch split-button to the Graph

3 participants